Skip to content

Conversation

AndyJado
Copy link
Contributor

@AndyJado AndyJado commented Sep 1, 2022

No description provided.

@rustbot rustbot added A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 1, 2022
@rust-highfive
Copy link
Contributor

r? @petrochenkov

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 1, 2022
@petrochenkov petrochenkov marked this pull request as ready for review September 1, 2022 14:28
@rustbot
Copy link
Collaborator

rustbot commented Sep 1, 2022

rustc_error_messages was changed

cc @davidtwco, @compiler-errors, @JohnTitor, @estebank, @TaKO8Ki

@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 1, 2022
@petrochenkov
Copy link
Contributor

Since this was submitted as a draft, changing it to S-waiting-on-author.

@davidtwco
Copy link
Member

r? @davidtwco

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@AndyJado
Copy link
Contributor Author

AndyJado commented Sep 3, 2022

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 3, 2022
@AndyJado AndyJado force-pushed the borrowck-diag-mig branch 2 times, most recently from 3f40403 to 5d2428f Compare September 4, 2022 08:31
@rust-log-analyzer

This comment has been minimized.

@AndyJado AndyJado force-pushed the borrowck-diag-mig branch 4 times, most recently from dc89bd1 to 2448a00 Compare September 5, 2022 08:16
@rust-log-analyzer
Copy link
Collaborator

The job mingw-check failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
    Checking rustc_codegen_ssa v0.0.0 (/checkout/compiler/rustc_codegen_ssa)
    Checking rustc_borrowck v0.0.0 (/checkout/compiler/rustc_borrowck)
    Checking rustc_mir_transform v0.0.0 (/checkout/compiler/rustc_mir_transform)
    Checking rustc_codegen_llvm v0.0.0 (/checkout/compiler/rustc_codegen_llvm)
error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:50:23
    |
50  |           let mut err = struct_span_err!(
50  |           let mut err = struct_span_err!(
    |  _______________________-
51  | |             self,
52  | |             new_loan_span,
53  | |             E0499,
...   |
56  | |             via(opt_via),
    | |_________- in this macro invocation
    |
note: the lint level is defined here
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:2:9
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:2:9
    |
2   | #![deny(rustc::diagnostic_outside_of_impl)]


error: diagnostics should be created using translatable messages
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:50:23
    |
50  |           let mut err = struct_span_err!(
50  |           let mut err = struct_span_err!(
    |  _______________________-
51  | |             self,
52  | |             new_loan_span,
53  | |             E0499,
...   |
56  | |             via(opt_via),
    | |_________- in this macro invocation
    |
note: the lint level is defined here
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:1:9
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:1:9
    |
1   | #![deny(rustc::untranslatable_diagnostic)]


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:177:23
    |
177 |           let mut err = struct_span_err!(
177 |           let mut err = struct_span_err!(
    |  _______________________-
178 | |             self,
179 | |             span,
180 | |             E0502,
...   |
187 | |             via(msg_old),
    | |_________- in this macro invocation


error: diagnostics should be created using translatable messages
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:177:23
    |
177 |           let mut err = struct_span_err!(
177 |           let mut err = struct_span_err!(
    |  _______________________-
178 | |             self,
179 | |             span,
180 | |             E0502,
...   |
187 | |             via(msg_old),
    | |_________- in this macro invocation


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 | macro_rules! struct_span_err {
    | ---------------------------- in this expansion of `struct_span_err!`
    | ---------------------------- in this expansion of `struct_span_err!`
609 |     ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |         $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:301:9
    |
    |
301 |         struct_span_err!(self, span, E0596, "cannot borrow {} as mutable{}", path, reason,)


error: diagnostics should be created using translatable messages
    |
608 | macro_rules! struct_span_err {
    | ---------------------------- in this expansion of `struct_span_err!`
    | ---------------------------- in this expansion of `struct_span_err!`
609 |     ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |         $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:301:9
    |
    |
301 |         struct_span_err!(self, span, E0596, "cannot borrow {} as mutable{}", path, reason,)


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:395:29
    |
395 |         self.infcx.tcx.sess.struct_span_err_with_code(sp, msg, code)


error: diagnostics should be created using translatable messages
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:395:29
    |
395 |         self.infcx.tcx.sess.struct_span_err_with_code(sp, msg, code)

error: could not compile `rustc_borrowck` due to 8 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `rustc_borrowck` due to 8 previous errors

@AndyJado
Copy link
Contributor Author

AndyJado commented Sep 9, 2022

@rustbot label +S-waiting-on-author -S-waiting-on-review

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 9, 2022
@AndyJado
Copy link
Contributor Author

moved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants